Skip to content

Comments

Definedness: Ret and Break are eager uses AST-65#205

Merged
nomeata merged 2 commits intomasterfrom
joachim/return-before-define
Mar 4, 2019
Merged

Definedness: Ret and Break are eager uses AST-65#205
nomeata merged 2 commits intomasterfrom
joachim/return-before-define

Conversation

@nomeata
Copy link
Contributor

@nomeata nomeata commented Mar 1, 2019

No description provided.

@nomeata nomeata changed the base branch from master to joachim/return-before-define-tests March 1, 2019 15:05
@nomeata nomeata requested a review from ggreif March 1, 2019 15:07
@nomeata nomeata force-pushed the joachim/return-before-define branch from c2fdf55 to 6058dc3 Compare March 1, 2019 15:12
@nomeata nomeata requested a review from crusso March 1, 2019 15:12
Copy link
Contributor

@crusso crusso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nomeata nomeata changed the base branch from joachim/return-before-define-tests to master March 3, 2019 11:10
@nomeata nomeata force-pushed the joachim/return-before-define branch from 6058dc3 to 512b0c8 Compare March 3, 2019 11:11
@nomeata
Copy link
Contributor Author

nomeata commented Mar 3, 2019

You did not click the magic button to approve :-)
(Or if you did, then me changing the base branch might have invalidated that.)

Copy link
Contributor

@ggreif ggreif left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes total sense. Let's see if the typo suggestions invalidate my approval when followed.

@ggreif
Copy link
Contributor

ggreif commented Mar 3, 2019

Just a question. When a (first) use happens in an eager context, after a break/return (i.e. it is a dead control flow), will that be marked as eager?

Co-Authored-By: nomeata <mail@joachim-breitner.de>
@nomeata
Copy link
Contributor Author

nomeata commented Mar 4, 2019

When a (first) use happens in an eager context, after a break/return (i.e. it is a dead control flow), will that be marked as eager?

Yes; after all the break might be conditional (inside an if). This analysis does not do dead code removal.

@nomeata nomeata merged commit ea899ca into master Mar 4, 2019
@nomeata nomeata deleted the joachim/return-before-define branch March 4, 2019 09:06
dfinity-bot added a commit that referenced this pull request Jan 12, 2021
mergify bot pushed a commit that referenced this pull request Jan 12, 2021
dfinity-bot added a commit that referenced this pull request May 7, 2021
mergify bot pushed a commit that referenced this pull request May 7, 2021
dfinity-bot added a commit that referenced this pull request Jun 24, 2023
## Changelog for ic-hs:
Branch: master
Commits: [dfinity/ic-hs@f092a2d5...c2cedde6](dfinity/ic-hs@f092a2d...c2cedde)

* [`f3acf563`](dfinity/ic-hs@f3acf56) attach cycles to canister creation in canister history tests ([dfinity/ic-hs⁠#203](https://github.com/dfinity/ic-hs/issues/203))
* [`f5c6e6ec`](dfinity/ic-hs@f5c6e6e) restart ic-ref in tests ([dfinity/ic-hs⁠#204](https://github.com/dfinity/ic-hs/issues/204))
* [`c2cedde6`](dfinity/ic-hs@c2cedde) update http_request fee calculation ([dfinity/ic-hs⁠#205](https://github.com/dfinity/ic-hs/issues/205))
mergify bot pushed a commit that referenced this pull request Jun 24, 2023
## Changelog for ic-hs:
Branch: master
Commits: [dfinity/ic-hs@f092a2d5...c2cedde6](dfinity/ic-hs@f092a2d...c2cedde)

* [`f3acf563`](dfinity/ic-hs@f3acf56) attach cycles to canister creation in canister history tests ([dfinity/ic-hs⁠#203](https://github.com/dfinity/ic-hs/issues/203))
* [`f5c6e6ec`](dfinity/ic-hs@f5c6e6e) restart ic-ref in tests ([dfinity/ic-hs⁠#204](https://github.com/dfinity/ic-hs/issues/204))
* [`c2cedde6`](dfinity/ic-hs@c2cedde) update http_request fee calculation ([dfinity/ic-hs⁠#205](https://github.com/dfinity/ic-hs/issues/205))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants